Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Expliciter la définiton des quêtes (PIX-16445) #11347

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

laura-bergoens
Copy link
Member

@laura-bergoens laura-bergoens commented Feb 5, 2025

🥞 Problème

Rendre plus explicite, dans la défintion de la quête, son fonctionnement.
Pour l’attestation, on s’attend à ce qu’un prescrit ait une participation pour chaque profil cible listé dans la collection fournie dans le requirement. Mais ce n’est pas clair dans la configuration de la quête, et ça a impliqué du code supplémentaire et difficile à lire pour décortiquer cette contrainte.

🥓 Proposition

Si on souhaite qu’un prescrit ait une campagne participation pour plusieurs profil cibles différents, en fait ça doit impliquer 1 requirement pour chaque participation. Il faut rendre ceci explicite dans la définition de la quête, et de fait ça exprimera le parcours pédagogique souhaité plus clairement dans la quête (et au passage ça simplifie un peu le code)

// avant
{
  type: campaignParticipations,
  data: {
     targetProfileIds: [1, 2],
  }
  comparison: all,
};

// après
[
  {
     type: campaignParticipations,
     data: {
      targetProfileId: 1,
     },
    comparison: all,
  }, 
  {
     type: campaignParticipations,
     data: {
      targetProfileId: 2,
     },
    comparison: all
  }, 
]

🧃 Remarques

Trucs à faire avant / pendant / après :

  • Attendre que les quêtes soient éditées dans le nouveau format
  • lancer le script de rattrapage d'attestation sur la durée entre le moment où les quêtes sont modifiées VS le déploiement de la MEP qui porte cette PR

😋 Pour tester

Non régression fonctionnelle sur l'obtention d'attestation

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@xav-car xav-car force-pushed the pix-16445-refacto-quests branch 2 times, most recently from 5919150 to 7d45a16 Compare February 5, 2025 17:12
@laura-bergoens laura-bergoens force-pushed the pix-16445-refacto-quests branch 2 times, most recently from 850de15 to d4b5566 Compare February 6, 2025 13:31
@laura-bergoens
Copy link
Member Author

Bloqué car on attend de passer la feature pour éditer les quêtes côté pixadmin et synchroniser l'ensemble

…le truc)

Co-authored-by: Yvonnick Frin <[email protected]>
Co-authored-by: Xavier Carron <[email protected]>
@laura-bergoens laura-bergoens force-pushed the pix-16445-refacto-quests branch from d4b5566 to 1588ce6 Compare February 6, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Func Review Needed Need PO validation for this functionally 👀 Tech Review Needed team-noa(h) noa - noah 🎾 ⚠️ Blocked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants